-
Notifications
You must be signed in to change notification settings - Fork 20
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Demand workflow #219
Demand workflow #219
Conversation
for more information, see https://pre-commit.ci
Yeah, please :) |
@hazemakhalek could you either add the UNSD to the |
@davide-f, will do |
@energyLS , will do. Also finishing up the projection feature. Will ping you when ready |
…the efficiency gains for the different sectors
for more information, see https://pre-commit.ci
for more information, see https://pre-commit.ci
Changed the demand scenario for the test config to include the energy_totals scripts in the test workflow
@davide-f I issued the PR for the new dependency. |
I'm updating the submodule to the latest version of the repo: #228 That should fix the issue once complete. I expect no breaking changes except the revision of the cutout name |
for more information, see https://pre-commit.ci
for more information, see https://pre-commit.ci
for more information, see https://pre-commit.ci
for more information, see https://pre-commit.ci
Next steps:
|
for more information, see https://pre-commit.ci
for more information, see https://pre-commit.ci
for more information, see https://pre-commit.ci
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
First testing shows: 2030 works, 2040 throws error, 2050 works but has the same values as 2030. What is the issue here?
for more information, see https://pre-commit.ci
for more information, see https://pre-commit.ci
|
@hazemakhalek I have checked this again, this error for e.g. appears due to missing config factors of e.g. fuell cell share. So this would just needed to be added to the config, but this could be done user specific and does not have to be part of the PR. One important thing though: since
|
@energyLS |
Fantastic, thanks! |
Closes # 79
Changes proposed in this Pull Request
This PR downloads the energy balances from UNSD, filters them by the chosen base year and country and processes the data to the sectors and carriers included in the model. The main output is: energy_totals for any country of the world for any historic year.
Following PR will project the energy_balances to the future (planning_horizons) year.
Checklist
envs/environment.yaml
andenvs/environment.docs.yaml
.config.default.yaml
,config.tutorial.yaml
, andtest/config.test1.yaml
.doc/configtables/*.csv
and line references are adjusted indoc/configuration.rst
anddoc/tutorial.rst
.doc/release_notes.rst
is amended in the format of previous release notes, including reference to the requested PR.